restart_dev.sh (1)

7-8: 하드코딩된 경로를 환경 변수로 변경하세요.

/home/ubuntu/caring-voice 경로가 하드코딩되어 있어 다른 환경에서 작동하지 않습니다.

다음과 같이 개선하세요:

-PROJECT_DIR="/home/ubuntu/caring-voice"
+PROJECT_DIR="${PROJECT_DIR:-$(pwd)}"
 cd $PROJECT_DIR

app/db_service.py (1)

92-113: 연결 사용자 조회 로직을 단일 쿼리로 최적화할 수 있습니다.

현재 구현은 보호자 조회 → 연결 사용자 조회 → 음성 조회로 3단계 쿼리를 실행합니다. 성능 향상을 위해 조인을 활용한 단일 쿼리로 리팩토링을 고려해보세요.

예시:

def get_care_voices(self, care_username: str, skip: int = 0, limit: int = 20) -> List[Voice]:
    """보호자(care)의 연결 사용자 음성 중 voice_analyze가 존재하는 항목만 최신순 조회"""
    from sqlalchemy.orm import joinedload, aliased
    
    CareUser = aliased(User)
    LinkedUser = aliased(User)
    
    q = (
        self.db.query(Voice)
        .join(LinkedUser, Voice.user_id == LinkedUser.user_id)
        .join(CareUser, CareUser.connecting_user_code == LinkedUser.user_code)
        .join(VoiceAnalyze, VoiceAnalyze.voice_id == Voice.voice_id)
        .filter(CareUser.username == care_username)
        .options(joinedload(Voice.questions), joinedload(Voice.voice_analyze))
        .order_by(Voice.created_at.desc())
        .offset(skip)
        .limit(limit)
    )
    return q.all()

app/voice_service.py (3)

81-83: 백그라운드 태스크의 실패를 추적할 수 없습니다.

asyncio.create_task로 생성된 태스크는 fire-and-forget 방식이므로 실패 시 호출자가 알 수 없습니다. 프로덕션 환경에서는 태스크 완료 상태와 에러를 추적할 수 있는 메커니즘(예: 로깅, 메트릭, 태스크 큐)을 고려하세요.

---

313-328: 예외 처리가 너무 광범위합니다.

모든 예외를 무시하고 빈 리스트를 반환하면 문제를 파악하기 어렵습니다. 최소한 로깅을 추가하거나 예외를 상위로 전파하는 것을 권장합니다.

     def get_care_voice_list(self, care_username: str, skip: int = 0, limit: int = 20) -> Dict[str, Any]:
         """보호자 페이지: 연결된 사용자의 분석 완료 음성 목록 조회(페이징)"""
         try:
             voices = self.db_service.get_care_voices(care_username, skip=skip, limit=limit)
             items = []
             for v in voices:
                 created_at = v.created_at.isoformat() if v.created_at else ""
                 emotion = v.voice_analyze.top_emotion if v.voice_analyze else None
                 items.append({
                     "voice_id": v.voice_id,
                     "created_at": created_at,
                     "emotion": emotion,
                 })
             return {"success": True, "voices": items}
-        except Exception:
+        except Exception as e:
+            print(f"get_care_voice_list error for {care_username}: {e}", flush=True)
             return {"success": False, "voices": []}

---

330-359: 예외 처리에 로깅을 추가하세요.

get_care_voice_list와 동일하게, 예외 발생 시 로깅을 추가하여 문제 추적을 용이하게 하세요.

     def get_user_voice_detail(self, voice_id: int, username: str) -> Dict[str, Any]:
         """voice_id와 username으로 상세 정보 조회"""
         try:
             voice = self.db_service.get_voice_detail_for_username(voice_id, username)
             if not voice:
                 return {"success": False, "error": "Voice not found or not owned by user"}
             # ... rest of the method
-        except Exception:
+        except Exception as e:
+            print(f"get_user_voice_detail error for voice_id={voice_id}, username={username}: {e}", flush=True)
             return {"success": False, "error": "Failed to fetch voice detail"}

app/main.py (1)

202-214: random 모듈이 중복 import되고 있습니다.

Line 27에서 이미 import random을 했는데 Line 208에서 다시 import하고 있습니다. 모듈 레벨 import를 사용하세요.

 @questions_router.get("/random")
 async def get_random_question():
     db = next(get_db())
     question_count = db.query(Question).count()
     if question_count == 0:
         return {"success": False, "question": None}
-    import random
     offset = random.randint(0, question_count - 1)

또한, 질문 테이블이 커질 경우 COUNT 쿼리의 성능을 고려하여 캐싱을 검토하세요.

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro